Skip to content

feat: print startup_time in final summary#185

Open
tykoo-chen wants to merge 1 commit intokarpathy:masterfrom
tykoo-chen:feat/print-startup-time
Open

feat: print startup_time in final summary#185
tykoo-chen wants to merge 1 commit intokarpathy:masterfrom
tykoo-chen:feat/print-startup-time

Conversation

@tykoo-chen
Copy link

Problem

startup_time is calculated (line 613) but never printed in the final summary. This information is useful for:

  • Understanding torch.compile overhead
  • Debugging slow startup issues
  • Calculating total experiment time vs training time

Solution

Add startup_seconds to the final summary output.

Example Output

---
val_bpb:          0.997900
training_seconds: 300.1
total_seconds:    325.9
...
depth:            8
startup_seconds:  25.8

startup_time is already calculated but not printed. Adding it to the
output helps understand compilation overhead and total experiment time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant